Add BBoxMaskPose to download count #1687
Conversation
MiraPurkrabek
requested review from
SBrandeis,
gary149,
Wauplin,
julien-c,
pcuenca and
ngxson
as code owners
| repoName: "BBoxMaskPose", | ||
| repoUrl: "https://github.com/MiraPurkrabek/BBoxMaskPose", | ||
| filter: false, | ||
| countDownloads: `path_extension:"pth"`, |
There was a problem hiding this comment.
Are all the three models used for inference or is one used at a time?
The reason I ask is, there are currently three different model checkpoint in the repo: https://huggingface.co/vrg-prague/BBoxMaskPose/tree/main
and the download counter will +=1 download counter for each hit, i.e. if all three are used for single inference, it'd result in +=3 which would make downloads inaccurate.
There was a problem hiding this comment.
Thank you for the review, @Vaibhavs10 .
Yes, all 3 models are needed to run the whole method -- it consists of their iterative usage and smart integration. However, all models could also work as standalone models if the user does not want to use them in the loop.
Therefore, it make sense to me to have all models in the same repo as they are connected to the same paper and together form the main method.
On the other hand, it also make sense to count downloads for each model. A user can download only one of the models and it should be counted.
Let me know if you see it differently or if I should change anything.
Thank you
Add BBoxMaskPose paper to download count.
HuggingFace link - BBoxMaskPose on HuggingFace
GitHub link - BBoxMaskPose on GitHub